Skip to content

Conversation

@ndr-ds
Copy link
Contributor

@ndr-ds ndr-ds commented May 2, 2025

Motivation

This state hash computation can apparently take much longer than 10ms

Proposal

Adjust the bucket sizes to be more generous and catch larger values

Test Plan

CI + have deployed networks with this already

Release Plan

  • Nothing to do / These changes follow the usual release cycle.

Copy link
Contributor Author

ndr-ds commented May 2, 2025

@ndr-ds ndr-ds force-pushed the 05-02-adjust_state_hash_computation_bucket branch from e83c215 to 3143c9d Compare May 5, 2025 21:00
@ndr-ds ndr-ds force-pushed the 05-02-actually_use_backend_config branch from fa5e6e5 to 40dc1bb Compare May 5, 2025 21:00
@ndr-ds ndr-ds force-pushed the 05-02-adjust_state_hash_computation_bucket branch from 3143c9d to a3d38ad Compare May 6, 2025 14:51
@ndr-ds ndr-ds force-pushed the 05-02-actually_use_backend_config branch 2 times, most recently from 0c1bef2 to 80c2534 Compare May 6, 2025 17:34
@ndr-ds ndr-ds force-pushed the 05-02-adjust_state_hash_computation_bucket branch from a3d38ad to 63ff628 Compare May 6, 2025 17:34
@ndr-ds ndr-ds force-pushed the 05-02-adjust_state_hash_computation_bucket branch from 63ff628 to cd3ab06 Compare May 7, 2025 17:10
@ndr-ds ndr-ds force-pushed the 05-02-actually_use_backend_config branch 3 times, most recently from 5395c3f to 93d807b Compare May 7, 2025 19:11
@ndr-ds ndr-ds force-pushed the 05-02-adjust_state_hash_computation_bucket branch from cd3ab06 to b7dd2db Compare May 7, 2025 19:11
@ndr-ds ndr-ds force-pushed the 05-02-adjust_state_hash_computation_bucket branch from b7dd2db to f6712ff Compare May 9, 2025 16:57
@ndr-ds ndr-ds force-pushed the 05-02-actually_use_backend_config branch from 93d807b to 8a865ae Compare May 9, 2025 16:57
Copy link
Contributor Author

ndr-ds commented May 9, 2025

Merge activity

  • May 9, 5:25 PM EDT: A user started a stack merge that includes this pull request via Graphite.
  • May 9, 5:27 PM EDT: Graphite rebased this pull request as part of a merge.
  • May 9, 5:28 PM EDT: @ndr-ds merged this pull request with Graphite.

@ndr-ds ndr-ds changed the base branch from 05-02-actually_use_backend_config to graphite-base/3877 May 9, 2025 21:25
@ndr-ds ndr-ds changed the base branch from graphite-base/3877 to main May 9, 2025 21:26
@ndr-ds ndr-ds force-pushed the 05-02-adjust_state_hash_computation_bucket branch from f6712ff to e54c74f Compare May 9, 2025 21:26
@ndr-ds ndr-ds merged commit 0bd0ef8 into main May 9, 2025
25 checks passed
@ndr-ds ndr-ds deleted the 05-02-adjust_state_hash_computation_bucket branch May 9, 2025 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants